Skip to content

Update industrial_ci action version in CI workflow#3734

Merged
nbbrooks merged 1 commit into
moveit:mainfrom
mosfet80:patch-27
May 25, 2026
Merged

Update industrial_ci action version in CI workflow#3734
nbbrooks merged 1 commit into
moveit:mainfrom
mosfet80:patch-27

Conversation

@mosfet80
Copy link
Copy Markdown
Contributor

@mosfet80 mosfet80 commented May 23, 2026

Description

Please explain the changes you made, including a reference to the related issue if applicable

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.28%. Comparing base (621b91d) to head (0a93024).

Additional details and impacted files
@@            Coverage Diff            @@
##           main    #3734       +/-   ##
=========================================
+ Coverage      0   46.28%   +46.28%     
=========================================
  Files         0      726      +726     
  Lines         0    59496    +59496     
  Branches      0     7625     +7625     
=========================================
+ Hits          0    27531    +27531     
- Misses        0    31798    +31798     
- Partials      0      167      +167     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mosfet80
Copy link
Copy Markdown
Contributor Author

@nbbrooks Fixed issue that caused ci-rolling-* tests to fail.

Copy link
Copy Markdown
Contributor

@nbbrooks nbbrooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like all that was needed for builds was ros-industrial/industrial_ci#928 - nice work!

@nbbrooks nbbrooks merged commit ab3ca28 into moveit:main May 25, 2026
11 of 12 checks passed
@github-project-automation github-project-automation Bot moved this to ✅ Done in MoveIt May 25, 2026
Comment thread .github/workflows/ci.yaml
- id: ici
name: Run industrial_ci
uses: ros-industrial/industrial_ci@ba2a3d0f830f8051b356711a8df2fedfc5d256cf
uses: ros-industrial/industrial_ci@c553397753252d630e88e7e91aa69c6c8a478ee0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The named commit sha refers to the master branch. So, why not using it directly?

Suggested change
uses: ros-industrial/industrial_ci@c553397753252d630e88e7e91aa69c6c8a478ee0
uses: ros-industrial/industrial_ci@master

@rhaschke
Copy link
Copy Markdown
Contributor

Fixed issue that caused ci-rolling-* tests to fail.

Note that industrial_ci just froze ROS Rolling on Noble to the latest supported index.
However, Rolling moved on to Resolute, which requires multiple packages to be newly released.
Thus, this fixes CI only on Noble, not on Rolling in general.

130s pushed a commit to 130s/moveit2 that referenced this pull request May 26, 2026
@nbbrooks
Copy link
Copy Markdown
Contributor

nbbrooks commented May 26, 2026

So this industrial ci change was a sort of "opt in" to rolling on resolute, but our configuration is still using the frozen noble version?

Screenshot_20260526-070131.png

@rhaschke
Copy link
Copy Markdown
Contributor

Yes, currently industrial_ci still uses Ubuntu Noble (and it's frozen Rolling index) when specifying ROS Rolling w/o a ubuntu release:
_ros2_defaults "noble"

To switch to the latest Rolling release on Ubuntu Resolute, one needs to specify the Ubuntu release via the environment variable OS_CODE_NAME.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants